-
Notifications
You must be signed in to change notification settings - Fork 757
[css-conditional-5] Defined @supports-condition #12622 #12935
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
I thought |
|
|
2ef2884 to
0fe9cfa
Compare
Oops, copy & pasta. 😅 I had first copied Lea's example and then changed the contents but not the name, while in the definition I chose "query", as that seems a little more consistent with "supports queries" and "media queries" and was also suggested by Lea. Anyway, I changed it to "supports condition" for now including all the related text. The PR includes an issue to bikeshed the name, anyway. @LeaVerou @mirisuzanne Could you please review the changes? Sebastian |
`@supports-query` is meant to allow authors to define and name custom support queries.
c97ea03 to
fa593d3
Compare
|
As noted in #12945, I had to rebase this PR. Therefore, re-requesting review. Sebastian |
|
Approved, but not merging yet in case @LeaVerou @mirisuzanne have further comments. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me
|
Thanks! Merging. |
|
[ moved to issue instead of here ] |
This defines the
@supports-queryat-rule that allows authors to create custom support queries. It closes #12622.Note that the resolution was to add it to the same spec as
@custom-media, i.e. Media Queries 5. Though, as noted in #12622 (comment), it is strongly tied to@supports. Therefore, I've added it to CSS Conditional 5, instead.Sebastian